Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes ordering for loaded events by API #81

Merged
merged 1 commit into from
Dec 1, 2023
Merged

Fixes ordering for loaded events by API #81

merged 1 commit into from
Dec 1, 2023

Conversation

butschster
Copy link
Member

This PR addresses an issue regarding the ordering of events after a page reload. Initially, all requested events are correctly fetched in descending order. However, a discrepancy arises when the frontend adds new events to the local store. Instead of inserting new events at the beginning of the array, which would maintain the descending order, they are appended to the end. This fix ensures that new events are correctly placed at the start of the array.

fixes #80

@butschster butschster added bug Something isn't working module:events labels Dec 1, 2023
@butschster butschster added this to the 1.0 milestone Dec 1, 2023
@butschster butschster self-assigned this Dec 1, 2023
@butschster butschster merged commit a194b59 into master Dec 1, 2023
@butschster butschster deleted the hotfix/80 branch December 1, 2023 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working module:events
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[BUG]. Events sorting datetime-desc
2 participants